Skip to content

Comments

Refactor test scaffolding for capability and recursion suites#58

Merged
mgomes merged 46 commits intomasterfrom
mgomes/refactor-tests
Feb 21, 2026
Merged

Refactor test scaffolding for capability and recursion suites#58
mgomes merged 46 commits intomasterfrom
mgomes/refactor-tests

Conversation

@mgomes
Copy link
Owner

@mgomes mgomes commented Feb 21, 2026

Why

The capability and recursion tests had repeated engine/compile/call boilerplate, which made updates noisy and harder to audit.

This refactor centralizes common test setup while preserving behavioral assertions and coverage.

What changed

  • Added shared test helpers for compile/call/error assertions.
  • Migrated context/events/db/jobqueue/foundations capability tests to helper-based structure.
  • Reduced repeated compile boilerplate in capability contract tests.
  • Refactored recursion tests to helper-based setup.

Validation

  • go test ./... (run after each atomic commit and once at the end)

@mgomes
Copy link
Owner Author

mgomes commented Feb 21, 2026

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Hooray!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@mgomes
Copy link
Owner Author

mgomes commented Feb 21, 2026

@codex review

@mgomes
Copy link
Owner Author

mgomes commented Feb 21, 2026

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Breezy!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@mgomes mgomes merged commit 8258105 into master Feb 21, 2026
5 checks passed
@mgomes mgomes deleted the mgomes/refactor-tests branch February 21, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant